Skip to content

wasm: update WAMR to WAMR-05-18-2022.#22157

Merged
htuch merged 3 commits intoenvoyproxy:mainfrom
PiotrSikora:wamr-2022-05-18
Jul 15, 2022
Merged

wasm: update WAMR to WAMR-05-18-2022.#22157
htuch merged 3 commits intoenvoyproxy:mainfrom
PiotrSikora:wamr-2022-05-18

Conversation

@PiotrSikora
Copy link
Copy Markdown
Contributor

Signed-off-by: Piotr Sikora piotrsikora@google.com

Signed-off-by: Piotr Sikora <piotrsikora@google.com>
@repokitteh-read-only
Copy link
Copy Markdown

As a reminder, PRs marked as draft will not be automatically assigned reviewers,
or be handled by maintainer-oncall triage.

Please mark your PR as ready when you want it to be reviewed!

🐱

Caused by: #22157 was opened by PiotrSikora.

see: more, trace.

@repokitteh-read-only repokitteh-read-only Bot added the deps Approval required for changes to Envoy's external dependencies label Jul 13, 2022
@repokitteh-read-only
Copy link
Copy Markdown

CC @envoyproxy/dependency-shepherds: Your approval is needed for changes made to (bazel/.*repos.*\.bzl)|(bazel/dependency_imports\.bzl)|(api/bazel/.*\.bzl)|(.*/requirements\.txt)|(.*\.patch).
envoyproxy/dependency-shepherds assignee is @htuch

🐱

Caused by: #22157 was opened by PiotrSikora.

see: more, trace.

Signed-off-by: Piotr Sikora <piotrsikora@google.com>
@PiotrSikora
Copy link
Copy Markdown
Contributor Author

Blocked on #22117, #22124 and #22154.

Copy link
Copy Markdown
Member

@htuch htuch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm deps

@repokitteh-read-only repokitteh-read-only Bot removed the deps Approval required for changes to Envoy's external dependencies label Jul 13, 2022
@htuch
Copy link
Copy Markdown
Member

htuch commented Jul 13, 2022

@PiotrSikora LMK when this is ready for final review. Thanks.
/wait

@PiotrSikora PiotrSikora marked this pull request as ready for review July 13, 2022 05:22
@PiotrSikora
Copy link
Copy Markdown
Contributor Author

@htuch it's part of a larger patchset with Wasm engine updates (#22117, #22124, #22154, #22157), so some tests are failing until preceding PRs are merged, but it should be ready for review, since I don't expect any code changes. If you want to take a look at all 4 PRs, that would be great. Thanks!

Copy link
Copy Markdown
Member

@htuch htuch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm deps

…-05-18

Signed-off-by: Piotr Sikora <piotrsikora@google.com>
@PiotrSikora
Copy link
Copy Markdown
Contributor Author

@htuch this is now good to go. Thanks!

@htuch htuch merged commit 8a37033 into envoyproxy:main Jul 15, 2022
phlax pushed a commit to phlax/envoy that referenced this pull request Nov 16, 2022
Signed-off-by: Piotr Sikora <piotrsikora@google.com>
phlax pushed a commit to phlax/envoy that referenced this pull request Nov 16, 2022
Signed-off-by: Piotr Sikora <piotrsikora@google.com>
phlax pushed a commit that referenced this pull request Nov 16, 2022
Signed-off-by: Piotr Sikora <piotrsikora@google.com>
phlax pushed a commit that referenced this pull request Nov 18, 2022
Signed-off-by: Piotr Sikora <piotrsikora@google.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants